Skip to content

test(vscode): port new Rslint E2E suites - #26

Merged
fi3ework merged 3 commits into
mainfrom
fix/lint-e2e-rslint-081
Aug 20, 2026
Merged

test(vscode): port new Rslint E2E suites#26
fi3ework merged 3 commits into
mainfrom
fix/lint-e2e-rslint-081

Conversation

@fi3ework

@fi3ework fi3ework commented Aug 20, 2026

Copy link
Copy Markdown
Member

Summary

  • port the upstream import/no-cycle VS Code E2E suite and fixtures
  • port generated rule-option type coverage for TypeScript and lint diagnostics
  • adapt the deprecated JSON config fixture to an equivalent native config while preserving upstream assertion semantics

Related Links

Checklist

  • Tests updated (or not required).
  • Documentation updated (or not required).

Testing

  • pnpm lint
  • pnpm test:unit
  • RSTACK_LINT_E2E_SUITES='Generated rule-option-types,import/no-cycle' VSCODE_CLI=1 pnpm test:e2e lint

Since @rslint/core 0.8.1 (web-infra-dev/rslint#1790), a file created after
its project was loaded is served by a type-info-less fallback Program until
a watcher event admits it into the configured project, so the first
non-empty publish may carry only non-type-aware rules on platforms with
slow file watchers (macOS). The fixAll suites create their targets through
withTmpFile and asserted on that first non-empty publish, which now fails.

Setup waits go through a new waitForDiagnosticsWithMessages helper that
waits until the expected diagnostics are present — the terminal assertion
is unchanged and holds on both 0.8.0 and 0.8.1. The lint fixture floor
moves to ^0.8.1 to match what CI resolves.
@fi3ework
fi3ework merged commit 96e08ae into main Aug 20, 2026
3 checks passed
@fi3ework
fi3ework deleted the fix/lint-e2e-rslint-081 branch August 20, 2026 09:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant